Skip to content

chore: Re-implementing tests (authzed-dotent tests are taken as template)#237

Open
radik wants to merge 5 commits into
authzed:mainfrom
radik:tests-refactoring
Open

chore: Re-implementing tests (authzed-dotent tests are taken as template)#237
radik wants to merge 5 commits into
authzed:mainfrom
radik:tests-refactoring

Conversation

@radik

@radik radik commented Jul 9, 2025

Copy link
Copy Markdown

Re-write of authzed-dotnet tests.
Solves #225

Existing tests are left for now, until review.

@github-actions

github-actions Bot commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@radik radik force-pushed the tests-refactoring branch from 403ea72 to 37df4d7 Compare July 10, 2025 17:41
@radik

radik commented Jul 10, 2025

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

authzedbot added a commit to authzed/cla that referenced this pull request Jul 10, 2025
@radik radik changed the title [WIP] chore: Re-implementing dotnet tests [WIP] chore: Re-implementing tests (authzed-dotent tests are taken as template) Jul 10, 2025
@radik radik marked this pull request as ready for review July 10, 2025 20:24
@radik radik requested a review from a team as a code owner July 10, 2025 20:24
@radik radik changed the title [WIP] chore: Re-implementing tests (authzed-dotent tests are taken as template) chore: Re-implementing tests (authzed-dotent tests are taken as template) Jul 10, 2025

@tstirrat15 tstirrat15 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it. Thank you!

@radik

radik commented Sep 6, 2025

Copy link
Copy Markdown
Author

@tstirrat15 great! How can I merge it?

@tstirrat15

tstirrat15 commented Sep 8, 2025

Copy link
Copy Markdown
Contributor

Existing tests are left for now, until review.

Was your plan to remove the other tests? I didn't want to merge if there was still something pending.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants